Skip to content

Conversation

@GitHK
Copy link

@GitHK GitHK commented Oct 5, 2024

What do these changes do?

Currently I'm stuck, waiting for service-library service to be properly fixed.

Related issue/s

How to test

Dev-ops checklist

@GitHK GitHK marked this pull request as draft October 5, 2024 11:13
@GitHK GitHK changed the title some changes WIP: porting dynamic-sidecar to pyndatic2 Oct 5, 2024
@GitHK GitHK marked this pull request as ready for review October 8, 2024 13:22
@GitHK GitHK requested a review from giancarloromeo as a code owner October 8, 2024 13:22
@GitHK GitHK changed the title WIP: porting dynamic-sidecar to pyndatic2 porting dynamic-sidecar to pyndatic2 Oct 10, 2024
Copy link
Owner

@giancarloromeo giancarloromeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot! Just a couple of comments from my side.

"application/json": {
"schema": {
"$ref": "#/components/schemas/ActivityInfo"
"anyOf": [
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you notice that now (it seems OpenAPI 3.1) the nullable fields have this new schema with anyOf and the new type "null"?

@giancarloromeo
Copy link
Owner

We should bring this one direct to pydantic_v2_migration.

@GitHK
Copy link
Author

GitHK commented Oct 15, 2024

closing in favour of ITISFoundation#6534

@GitHK GitHK closed this Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants